Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#142 [feat] 버전에 따른 다이얼로그 #149

Merged
merged 11 commits into from
Feb 1, 2024

Conversation

stellar-halo
Copy link
Contributor

@stellar-halo stellar-halo commented Jan 31, 2024

📑 Work Description

  • 버전에 따라 스토어로 보내는 다이얼로그를 띄우는 작업을 했습니다!
  • api 연결을 했는데, header에 accessToken이 없어서 모듈을 하나 더 만들어줬습니다. (씁)
  • 겸사겸사 usecase 들을 폴더링 했습니다. (본인의 것이 다르게 된 것이 있다면 알려주세요)

🛠️ Issue

📷 Screenshot

image
image

💬 To Reviewers

  • 보일러 플레이트가 많아서 거슬리네요..

@stellar-halo stellar-halo added Pull Request pr 날림! Feat 새로운 기능 추가 강희🐬 강희가 작업함! labels Jan 31, 2024
@stellar-halo stellar-halo self-assigned this Jan 31, 2024
Copy link
Member

@minemi00 minemi00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

버전 업데이트 어렵네요.. 수고하셨습니다!

Comment on lines +77 to +78
private const val MAXIMUM_STORE_VERSION = "999.9.9"
private const val MAXIMUM_FORCE_VERSION = "999.0.0"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㄷㄷ 999버전까지 가보자고...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

근데 이건 정말 단순 궁금한건데.. store 버전은 999.9.9까지고 force 버전은 999.0.0인 이유가 있나여...??

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

별 이유는 없고 만약에 버전 서버 통신 이슈로 설정해둔 값이 들어가게 되면 유연한or강제 업데이트가 띄워지면 좋겠어서 했습니다!

import javax.inject.Qualifier
import javax.inject.Singleton

@Module
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

모듈까지 새로 만드느라 고생하셧슴다,!

Copy link
Member

@emjayMJkim emjayMJkim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니당!!!

@stellar-halo stellar-halo requested a review from m1njae February 1, 2024 10:12
Copy link
Member

@m1njae m1njae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

릴리즈 가보자고~

@stellar-halo stellar-halo merged commit a10e00f into develop Feb 1, 2024
1 check passed
@stellar-halo stellar-halo deleted the feature/#142-link-store-depend-version branch February 1, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 추가 Pull Request pr 날림! 강희🐬 강희가 작업함!
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[feat] 버전 관리
4 participants